From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Tue, 16 Feb 2021 02:56:09 +0000 (+0100)
Subject: gdm: Override any other lower-priority service message on error
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~616^2^2~319
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=b74900b3a3dd7cd6484255e2ef209e3a65507294;p=gnome-shell.git

gdm: Override any other lower-priority service message on error

When we got an error, all the other HINT or INFO messages are not useful
anymore and delaying to show them is just a waste of time and may be
even wrong in scenarios with fast authentication devices.

An example are the fingerprint devices, where the user may touch the
sensor repeatedly while we may still show the "touch the sensor" hint
instead of notifying of possible errors.

So, in case we got an error override all the other errors coming from
the same service with lower priority.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1683>
---

diff --git a/js/gdm/util.js b/js/gdm/util.js
index f1c3c9149..1ee84acde 100644
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -603,7 +603,8 @@ var ShellUserVerifier = class {
         if (!this.serviceIsForeground(serviceName) && !isFingerprint)
             return;
 
-        this._queueMessage(serviceName, problem, MessageType.ERROR);
+        this._queuePriorityMessage(serviceName, problem, MessageType.ERROR);
+
         if (isFingerprint) {
             // pam_fprintd allows the user to retry multiple (maybe even infinite!
             // times before failing the authentication conversation.